Fix initial notification data lost issue and empty foreground notification issue#496
Conversation
|
Can confirm this fixed the empty foreground notification issue for me. Thanks! |
|
i have improved your fork. but it's not change anything |
|
Experiencing the same issue. Would be really nice if this can be merged :) |
|
I did try to implement the changes of this PR, but afterwards the push notifications didn't work anymore. Did you tested this properly @patrickyin? |
|
works for me |
|
when this will be merged ? |
|
I am using one signal with this library, and had to make different (but similar) changes to the props file. I think a better solution than this PR would be the ability to set the key name to find the body and title inside javascript (defaulting to body/title). That way you could get it working with any payload format. |
|
+1 |
|
How can i install this? Can copy package.json line please? |
|
@luisfuertes I used patch-package, example patch and it's worked well |
|
Thx for answer @doodlemoonch , i didnt know that component! Finally i install with: Now in foreground, notification title and body are shown. But notification icon continue broken. EDIT: To fix icon i only have to renamed icon png file to 'notification_icon' |
|
please have this merged |
|
Hmm, since I don't have permission to merge this PR, it might need code owner's help. @yogevbd |
|
it worked for me too. Can you merge this @yogevbd ? |
|
yeah, it worked for me! @yogevbd |
|
I'm running The issue is also reported here: #615 Any ideas what could be missing? |
|
Interestingly I've been sending notifications using the AWS Pinpoint console, using the built in "Campaigns" functionality. I noticed that in the adb logs the body and title appeared in I wonder if this is normal and if the proper body and title could be used automatically somehow for all sorts of notifications, regardless of how they're sent. Granted, I'm not very familiar with Android notifications, but on iOS this issue doesn't exist. |
This PR fixes:
titleandbodyfiles undernotificationfield rather thandata.undefinedinitial notification data if your notification payload hastitleandbodyfiles undernotificationfield rather thandata.Details of issue and notification payloads: #456 (comment)
Related issues: #456 , #492